Skip to content

Sayali: add weekly PR grading screen with backend API integration and inline summary#5164

Open
sayali-2308 wants to merge 1 commit intodevelopmentfrom
Sayali_Weekly_PR_Grading_Frontend
Open

Sayali: add weekly PR grading screen with backend API integration and inline summary#5164
sayali-2308 wants to merge 1 commit intodevelopmentfrom
Sayali_Weekly_PR_Grading_Frontend

Conversation

@sayali-2308
Copy link
Copy Markdown
Contributor

image

Description

Implements #12 (Priority Urgent)

Related PRs:

This frontend PR is related to backend PR #3826 (already merged).

Main changes explained:

  • Updated PRGradingScreen.jsx to fetch grading data from backend API on load via useEffect
  • Updated PRGradingScreen.jsx to save grades to backend when Done is clicked with saving/success feedback
  • Fixed grade enum mismatch (Cannot find imageNo Correct Image) to match backend
  • Redesigned summary section to display inline below each reviewer's row (matching mockup)
  • Added interactive inline checkboxes allowing grades to be updated from both PR number tags and inline summary
  • Created weeklyGradingActions.js with getWeeklyGrading and saveWeeklyGrading API action functions
  • Added WEEKLY_GRADING and WEEKLY_GRADING_SAVE endpoints to src/utils/URL.js

How to test:

  1. Check out branch Sayali_Weekly_PR_Grading_Frontend
  2. npm install and npm run start:local
  3. Clear site data/cache
  4. Log as admin user
  5. Navigate to http://localhost:5173/pr-grading-test
  6. Click on any team (e.g. 91NePRT)
  7. Verify table shows Reviewer Name, PR reviewed, PRs Needed, PR Numbers columns
  8. Click + Add new and enter a valid PR number (e.g. 1070 or 1070 + 1256)
  9. Verify pair PRs show in green, single PRs in grey
  10. Verify inline summary appears below each reviewer with interactive checkboxes
  11. Change a grade from inline summary and verify it updates
  12. Click Done and verify saving feedback appears
  13. Verify dark mode works correctly

Screenshots or videos of changes:

image image image image image

Note:

Backend is already implemented in merged PR #3826. This PR only covers frontend integration.

@netlify
Copy link
Copy Markdown

netlify bot commented Apr 17, 2026

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit 9299416
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/69e286cafdc6570008847f3e
😎 Deploy Preview https://deploy-preview-5164--highestgoodnetwork-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@sonarqubecloud
Copy link
Copy Markdown

@sayali-2308 sayali-2308 added the High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible label Apr 17, 2026
Copy link
Copy Markdown

@Rajasrivatsansrinivasan Rajasrivatsansrinivasan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed PR #5164 by Sayali. Checked out the branch locally and tested the weekly PR grading screen with the integrated backend flow. Verified that team selection loads reviewer grading data correctly, the table shows the expected columns, new PR numbers can be added, pair and single PR styling behaves as expected, inline summary checkboxes are interactive, and clicking Done saves the grading updates with visible feedback. Also confirmed the page remains usable in dark mode. Approving.

Image Image Image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants